Skip to content

Refactor remove country from search and headline details page #30

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

fulleni
Copy link
Member

@fulleni fulleni commented May 31, 2025

Description

Type of Change

  • ✨ New feature (non-breaking change which adds functionality)
  • 🛠️ Bug fix (non-breaking change which fixes an issue)
  • ❌ Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 Code refactor
  • ✅ Build configuration change
  • 📝 Documentation
  • 🗑️ Chore

fulleni added 15 commits May 31, 2025 19:00
- Removed unnecessary bloc
- Simplified account creation flow
- Removed country follow pages
- Removed country repository usage
- Removed country search option
- Simplified model type handling
- Adjusted UI and logic to match
- Removed country search type
- Updated display names
- Removed country repository
- Removed country search logic
- Removed headquarters chip
- Fixed non-null assertion operator usage
- Refactor UI for clarity
- Add loading and error states
- Improve navigation
- Use shared widgets
- Page moved to a shared module
- Display followed categories
- Allows unfollowing categories
- Navigates to add more categories
- Removed countries list tile
- Simplified followed items page
- Handle loading and failure states
- Load prefs on user change in AccountBloc
- Corrected event name
- Refactor event and state names
- Improve error handling
- Remove unused code
- Add clear preferences event
@fulleni fulleni merged commit 7000630 into main May 31, 2025
0 of 2 checks passed
@fulleni fulleni deleted the refactor_remove_country_from_search_and_headline_details_page branch May 31, 2025 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant